Skip to content

feature/issue-28/implement-batch-processing#36

Merged
alperkent merged 10 commits into
mainfrom
alperkent/issue28
Jul 30, 2025
Merged

feature/issue-28/implement-batch-processing#36
alperkent merged 10 commits into
mainfrom
alperkent/issue28

Conversation

@alperkent
Copy link
Copy Markdown
Collaborator

Resolves #28.

…coefficient_of_variation' and update documentation for consistency
… function and integrate its functionality into extract_features. Update tests to reflect changes in feature extraction process.
@alperkent alperkent added enhancement New feature or request refactor Alterations of code that do not affect function task A development task intended for Github Projects labels Jul 22, 2025
@alperkent alperkent self-assigned this Jul 22, 2025
@alperkent alperkent requested a review from Asanto32 July 22, 2025 15:44
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (ba62c7b) to head (3e5783e).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #36   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          305       346   +41     
=========================================
+ Hits           305       346   +41     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alperkent
Copy link
Copy Markdown
Collaborator Author

@Asanto32 implemented the changes you suggested

Comment thread tests/unit/test_orchestrator.py Outdated
…t errors and update test_orchestrator.py to remove commented-out tests
Copy link
Copy Markdown
Collaborator

@Asanto32 Asanto32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small comments about potentially redundant tests, but overall lgtm!

Comment thread src/graphomotor/core/orchestrator.py Outdated
Comment thread src/graphomotor/core/orchestrator.py Outdated
Comment thread tests/unit/test_orchestrator.py
Comment thread tests/unit/test_orchestrator.py
Comment thread tests/unit/test_orchestrator.py Outdated
Comment thread tests/unit/test_orchestrator.py Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@alperkent alperkent merged commit f66a787 into main Jul 30, 2025
25 checks passed
@alperkent alperkent deleted the alperkent/issue28 branch July 30, 2025 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request refactor Alterations of code that do not affect function task A development task intended for Github Projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Task: Add batch processing capability

3 participants